-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(Freek): add presence #9014
Conversation
websites/F/Freek/metadata.json
Outdated
"manga" | ||
], | ||
"iframe": true, | ||
"iFrameRegExp": ".*", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you able to use a more narrow regex?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not really sure what it iframe regex is. Is it just limiting the domain sources of the iframe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, right now .*
(meaning anything), will make the iframe.js script load into all iframes, regardless of URL.
It would be nice if you could limit it to a certain domain or path for example, so it doesn't load into Google Analytics iframe for example.
Head branch was pushed to by a user without write access
Description
This pr implements the following:
Support for current pages:
Unsupported (coming soon):
Acknowledgements
yarn format
Screenshots
Watching a movie
Watching a tv show
Searching
Exploring
Private browsing
History
Manga